Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Start of geospatial analysis #21

Merged
merged 15 commits into from
Jan 25, 2024
Merged

Start of geospatial analysis #21

merged 15 commits into from
Jan 25, 2024

Conversation

barneydobson
Copy link
Collaborator

@barneydobson barneydobson commented Jan 19, 2024

Description

Create a module to migrate code to perform various geospatial operations on rasters and shapes

Fixes part of #19

Key features

  • Raster interpolation
  • Reprojections and projection finding
  • Country finding
  • Nearest nodes
  • Carve

-Add raster interpolation and tests
@barneydobson barneydobson self-assigned this Jan 19, 2024
@barneydobson
Copy link
Collaborator Author

barneydobson commented Jan 22, 2024

Would @dalonsoa and @cheginit want to review PRs in the initial migration? or just further down the line

Copy link
Collaborator

@dalonsoa dalonsoa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've made a couple of minor comments, but it looks good, otherwise.

One of the checks is failing because the link checker cannot find some links, in particular those that point to somewhere private (this repo is private). You can disable the checking them with this https://github.com/marketplace/actions/markdown-link-check#disable-check-for-some-links

swmmanywhere/geospatial_operations.py Outdated Show resolved Hide resolved
swmmanywhere/geospatial_operations.py Outdated Show resolved Hide resolved
swmmanywhere/geospatial_operations.py Outdated Show resolved Hide resolved
swmmanywhere/geospatial_operations.py Outdated Show resolved Hide resolved
@barneydobson barneydobson merged commit ec77b84 into main Jan 25, 2024
10 checks passed
@barneydobson barneydobson deleted the geospatial_analysis branch January 26, 2024 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants